Add diverse tests for TRA #66
Conversation
|
Also, please run the PR checks for verification |
|
@Baaryan I don't think I have the permission to run the above PR checks. Could you re-trigger them? |
jest.config.js
Outdated
| maxWorkers: 5, | ||
| roots: ["src"], | ||
| testMatch: ["**/*.test.js"], | ||
| testMatch: ["**/test*.js"], |
There was a problem hiding this comment.
Older tests would should running right?
There was a problem hiding this comment.
Older tests are not currently needed. We are planning to add new tests altogether.
jest.config.js
Outdated
| testMatch: ["**/test*.js"], | ||
| testPathIgnorePatterns: ["/node_modules/"], | ||
| testTimeout: 60 * 1000, | ||
| testTimeout: 30 * 1000, |
There was a problem hiding this comment.
Why is test timeout reduced?
There was a problem hiding this comment.
test timeout of 30 seconds also works. 60 seconds was a bit high and was extending the duration of the build
package.json
Outdated
| "scripts": { | ||
| "sample-test": "browserstack-node-sdk jest src/sample.test.js", | ||
| "sample-test": "browserstack-node-sdk jest 'src/module[A-Z]/test.js'", | ||
| "sample-local-test": "browserstack-node-sdk jest src/sample-local.test.js", |
There was a problem hiding this comment.
When we try to run the sample-local-test it will fail since above regex is modified.
There was a problem hiding this comment.
Changed the config to match all tests
4 participants
No description provided.